Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix bugs and enhance unit test coverage for all functions in pkg/util/validator.go #4505

Merged
merged 3 commits into from
Sep 18, 2024

Conversation

dolibali
Copy link
Contributor

Pull Request

What type of this PR

Examples of user facing changes:

  • Bug fixes
  • Tests

Which issue(s) this PR fixes

Fixes #4504

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. bug Something isn't working go Pull requests that update Go code test automation tests labels Sep 11, 2024
@dolibali dolibali changed the title fix bugs in validator.go and enhance unit test coverage for all functions fix bugs and enhance unit test coverage for all functions in pkg/util/validator.go Sep 11, 2024
@oilbeater
Copy link
Collaborator

@dolibali please fix the DCO issue

pkg/util/validator.go Outdated Show resolved Hide resolved
pkg/util/validator.go Outdated Show resolved Hide resolved
pkg/util/validator.go Outdated Show resolved Hide resolved
pkg/util/validator_test.go Outdated Show resolved Hide resolved
pkg/util/validator_test.go Outdated Show resolved Hide resolved
pkg/util/validator_test.go Outdated Show resolved Hide resolved
pkg/util/validator_test.go Outdated Show resolved Hide resolved
pkg/util/validator_test.go Outdated Show resolved Hide resolved
pkg/util/validator_test.go Outdated Show resolved Hide resolved
pkg/util/validator_test.go Outdated Show resolved Hide resolved
pkg/util/validator_test.go Outdated Show resolved Hide resolved
pkg/util/validator_test.go Outdated Show resolved Hide resolved
pkg/util/validator_test.go Outdated Show resolved Hide resolved
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 18, 2024
@bobz965 bobz965 merged commit a2b22ec into kubeovn:master Sep 18, 2024
9 of 10 checks passed
zcq98 pushed a commit to zcq98/kube-ovn that referenced this pull request Sep 18, 2024
…/validator.go (kubeovn#4505)

* Fix bugs in validator.go and enhance unit test coverage for all functions

---------

Signed-off-by: dolibali <[email protected]>
Co-authored-by: dolibali <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working go Pull requests that update Go code lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files. test automation tests
Projects
None yet
4 participants